fix: use only php min and max in linter#657
Conversation
Signed-off-by: SebastianKrupinski <krupinskis05@gmail.com>
|
Instead we will switch linting to take 1-2 seconds, then this is not worth it anymore: I'm not aware there was ever a linting issue on a middle version, but not worth it from my POV |
I made the improvement in the calendar application to reduce the amount of jobs that need to run, one to the team members recommended I share the change so that other apps can adopt it. If this not worth it we can close the PR |
Actually disregard we can close this... I think it is still valid... It still takes time, processing power and bandwith to setup a runner, download the images and run the tests... so I think this is still valid change |
susnux
left a comment
There was a problem hiding this comment.
Agree with this, we should only schedule CI jobs we really need.
|
This is really the wrong seconds saved, but sure. |
Summary